Conversation
|
@nullableVoidPtr thx for your work! Please let me know when you think the code is ready for reviewer! |
|
The push / pop lifting looks fine on the binary https://github.com/user-attachments/files/23866882/apxbin.zip |
|
Thanks for the test bin - I'll be sure to test against that; do you have a corpus for the other extension instructions? |
|
👏 |
I do not, I will make another with more of the extensions instructions expressed, thank you for responding quick! |
|
Heya, I consider this extension complete - is there any other blockers besides your testing? |
None, just have not merged it yet, will get it done shortly, sorry! |
|
The sizing of non-ndd instructions was messed up between commits, I am running tests on the branch below, if you could double check my changes, once im done what I will do is merge my changes into your branch and then squash with you as the author |
|
Good catch with the NDD op lens, LGTM |
|
Okay testing is done, I checked this branch against ~200k object files and nothing changed (which is good!), no performance regressions either, however I did find a blind spot in our COFF relocations in one binary leading to a fall through of a no-return call that then disassembles as an NDD instruction, but that is a separate issue entirely 😆. |
|
Available in |

Resolves #5246